-
-
Notifications
You must be signed in to change notification settings - Fork 514
Feature/organization improvements #1930
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
niemyjski
wants to merge
48
commits into
main
Choose a base branch
from
feature/organization-improvements
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
+2,959
−1,145
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Uses HashSet instead of Collection for storing user organization IDs and roles. This change improves performance by leveraging the efficient lookup capabilities of HashSet, ensuring uniqueness and faster access when checking for organization or role membership.
Adds links to manage organization settings and billing directly from the user and organization switcher dropdowns. This provides quick access to organization management features, improving the user experience.
Corrects user pagination by including pending invites in the total user count for accurate "has more" calculations.
Replaces the `goto` function calls with the `A` component from the typography module for navigating to organization management and creation pages. This change enhances the user experience and allows for styling.
Updates Svelte and Typescript-eslint dev dependencies. This ensures the project is using the latest available versions of these packages, which may include bug fixes, performance improvements, and new features.
Refactors code to use the TimeProvider abstraction when introducing delays. This allows for easier testing and mocking of time-dependent operations in jobs and migrations. Also changes AllScopes to be an ISet.
Ensures the notification settings form only renders after the form data is available. This prevents errors that occur when attempting to access properties of an undefined object.
Refactors the Beast Mode chatmode configuration for improved tool usage and instruction clarity. The update includes a refined set of available tools, clarifies internet research guidelines with more specific instructions on link fetching, and emphasizes the importance of memory management to avoid redundant file reads. Also enforces .env file creation and updates todo list formatting.
Refactors notification settings forms for users and integrations, improving UI and separating concerns. Introduces a generic `UserNotificationSettingsForm` for user-specific settings and an `IntegrationNotificationSettingsForm` for integration settings, enhancing code organization and reusability. Updates the UI to use a consistent card-based layout with descriptive text for each setting.
Enhances the user interface for the user notification settings form. Replaces the heading component to ensure consistent styling. Improves the loading state of the form by using a more visually appealing skeleton loader, providing a better user experience while data is being fetched.
Enhances the appearance settings page layout by aligning the theme options (Light, Dark, System) to the center using flexbox. This provides a better user experience by visually centering the theme previews and labels.
Updates dependencies to their latest versions. This ensures the project benefits from the latest features, bug fixes, and security patches in the updated libraries.
Refactors the notification settings forms for projects and users. This change introduces a new `UserNotificationSettingsForm` component that provides a better UI, including descriptions for each notification type. It also handles the display of settings based on plan features and email verification status.
Refactors the alert component to use variants for styling, and applies the 'information' variant to the alert in the user notification settings form to enhance readability.
Improves reactivity by utilizing SvelteMap for filter caching and SvelteDate for date calculations, enhancing UI responsiveness.
Refactors the notification settings form to use `onCheckedChange` instead of `onclick` for switch components, ensuring consistent event handling. Moves `isEmailAddressVerified` declaration for better readability. Improves project notification settings saving with toast notifications to provide user feedback.
Introduces `structuredCloneState` utility to prevent cache mutation and reactive entanglement when working with forms. Updates form components to use `structuredCloneState` for initializing and resetting form data, ensuring data isolation and predictable behavior. Adds Svelte component guidelines to promote best practices and consistent development. Improves performance by using reference comparison instead of JSON stringification for reset logic. Also adds reactive binding pattern to handle query data.
Adds documentation for async components in Svelte. Enables the `experimental.async` compiler option in the Svelte configuration file to allow using `await` directly in component scripts, `$derived` expressions, and template expressions.
Updates various dependencies to their latest versions. This includes updates to `@lucide/svelte`, `bits-ui`, `shiki`, `tailwind-variants`, `@chromatic-com/storybook`, `@iconify-json/lucide`, `@playwright/test`, `@storybook/*`, `@sveltejs/kit`, `@types/node`, `eslint-plugin-storybook`, `svelte`, `svelte-check`, `typescript`, and `typescript-eslint/*`. These updates may include bug fixes, performance improvements, new features, and security patches from the updated libraries.
Adds documentation outlining best practices and patterns for frontend development, including dialog component structure, API integration, naming conventions, and type safety. These guidelines aim to promote consistency, maintainability, and code quality across the frontend codebase.
Updates the component name from "DropDown" to "Dropdown" for consistency and to adhere to naming conventions. This change affects the component itself, its import statements, and its usage within the application.
Updates the organization API to use organization ID instead of email for user operations. Changes `removeOrganizationUser` to `deleteOrganizationUser` and updates related components. Renames `UpdateOrganizationRequest` to `PatchOrganizationRequest` and updates the corresponding API call to use PATCH.
Updates the placeholder text in the set event bonus and suspend organization dialogs. This improves clarity and user experience by providing more helpful hints.
Adds documentation for using Superforms within dialogs. The new documentation covers specific patterns for handling server-side validation, preventing focus theft by SvelteKit, and ensuring proper error propagation. It also emphasizes the importance of rethrowing errors from action functions called within dialogs.
Bumps various dependency versions, including `@lucide/svelte`, `@sveltejs/adapter-static`, `@sveltejs/kit`, `bits-ui`, and `svelte`. Also, adds "unsuspended" to the VS Code settings.
Improves error handling in the set bonus organization dialog. Catches validation exceptions from the server and displays them in the form. Also, fixes an issue where the expiration date was not being correctly converted to UTC.
Improves error handling in dialog forms by applying server-side validation errors and displaying them to the user. Also, throws errors from async mutations to be caught by the parent component.
Enhances the admin controller by implementing improved validation for organization IDs and providing more descriptive error messages. The changes introduce model validation for the `SetBonusAsync` action, ensuring that the organization ID is valid before proceeding. This prevents unexpected errors and provides better feedback to the user. Additionally, it adds API documentation to the SetBonusAsync method. The change also fixes an issue in the `AuthController` where the change password method was not returning a validation error when the current password was incorrect.
Updates and reorganizes project instructions, removing the outdated 'Beast Mode' chatmode and improving the structure of the main copilot instructions. Specifically: - Removes the obsolete 'Beast Mode' chatmode file. - Moves the 'Key Principles' section to the top of `copilot-instructions.md` for better visibility. - Simplifies backend testing instructions.
This change adds a unique ID to each superform instance. This will allow us to properly target and interact with forms, enabling improved testing and accessibility.
Handles cases where a facet filter builder is not found by returning `undefined` to prevent errors and then filters out any undefined values to prevent rendering issues.
Removes the pie chart card component as it is no longer needed.
Replaces the card components with typography components (H3, Muted) to improve visual consistency and styling control across the application. This change ensures a more uniform and maintainable UI by leveraging the typography components for titles and descriptions.
Adds unique IDs to Superforms for easier identification and manipulation, particularly when dealing with multiple forms on a single page or when needing to target specific forms via JavaScript. This enhancement improves form management and accessibility.
Removes the key block surrounding the children render. This change avoids potential issues related to state management and component re-renders when the route changes.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.